Skip to content

fix: resolve tsconfig paths with tsx #68

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
wants to merge 8 commits into
base: main
Choose a base branch
from

Conversation

zirkelc
Copy link
Contributor

@zirkelc zirkelc commented Jun 24, 2025

Fixes #60

This PR replaces jiti with tsx to load the serverless.ts config because tsx resolves tsconfig.paths while jiti does not yet (unjs/jiti#373)

I will try to add a new test case with a tsconfig.json and paths.

@zirkelc
Copy link
Contributor Author

zirkelc commented Jun 25, 2025

Sorry, I didn't want to push my local lockfile and the incomplete test.

@mnapoli
Copy link
Contributor

mnapoli commented Jun 25, 2025

Should we remove the jiti dependency too?

@zirkelc
Copy link
Contributor Author

zirkelc commented Jun 25, 2025

@mnapoli sure!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants